Skip to content

Conversation

@dumbbell
Copy link
Collaborator

This pull request fixes small unrelated issues with the testsuites that cause local or CI failures.

@dumbbell dumbbell self-assigned this Oct 16, 2025
…d modules

[Why]
rabbit_access_control checks that the auth backend module is provided by
`rabbit` or an enabled plugin. Some testsuites use a local auth backend
module for tests purpose that do not come as a plugin. We need a
mechanism to allow that.

[How]
We introduce a new `test_auth_backends` application envoironment
variable that testsuites can used to declare their valid auth backend
modules.
[Why]
rabbit_access_control now ensures all configured auth backend modules
are provided by `rabbit` or an enabled plugin. This is not the case for
this testsuite's own auth backend.

[How]
Declare `rabbit_auth_backend_mqtt_mock` as a test auth backend module.
[Why]
It didn't handle them before and crashed later when it assumed the
return value was a list.
…ch()`

[Why]
Hopefully, this fixes some test flakes in CI.
@dumbbell dumbbell force-pushed the set-of-small-fixes-here-and-there branch from 452d571 to 274ef7d Compare October 17, 2025 08:17
@dumbbell dumbbell marked this pull request as ready for review October 17, 2025 08:59
@dumbbell dumbbell merged commit 6515471 into main Oct 17, 2025
291 checks passed
@dumbbell dumbbell deleted the set-of-small-fixes-here-and-there branch October 17, 2025 08:59
@dumbbell dumbbell added this to the 4.3.0 milestone Oct 17, 2025
dumbbell added a commit that referenced this pull request Oct 17, 2025
Collection of test fixes (2025Q4) (backport #14742)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants